Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the documentation regarding Let's Encrypt #2051

Merged
merged 1 commit into from
Apr 5, 2022

Conversation

misilot
Copy link
Contributor

@misilot misilot commented Feb 11, 2022

With the additions from Islandora-Devops/isle-dc#228 requesting
certificates via ACME / Let's Encrypt has changed, and should be easier
for users to utilize.

Purpose / why

Updated the documentation relating to requesting SSL Certificates through Let's Encrypt and Traefik.
https://islandora.github.io/documentation/installation/docker-custom/#using-lets-encrypt-instead-of-certificate-files

What changes were made?

Updated text to reflect only having to change environment variables in the .env file.

Verification

If Islandora-Devops/isle-dc#228 gets merged, ACME Certificate generation is controlled by a few simple variables. Also debugging Traefik would be also controlled by a variable.

Interested Parties

@Islandora/documentation
@Islandora/8-x-committers_


Checklist

Pull-request reviewer should ensure the following

Person merging should ensure the following

  • Does mkdocs still build successfully? (This is indicated by TravisCI passing. To test locally, and see warnings, see How To Build Documentation.)
  • If pages are renamed or removed, have all internal links to those pages been fixed?
  • If pages are added, have they been linked to or placed in the menu?
  • Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

@manez
Copy link
Member

manez commented Mar 9, 2022

Discussed at DIG that there was a CLA submitted, so this is covered. @misilot will resend to make sure it gets filed properly, but it can be considered ok for merging purposes 😄

@ysuarez ysuarez self-requested a review March 16, 2022 17:58
@ysuarez
Copy link
Contributor

ysuarez commented Apr 5, 2022

With Islandora-Devops/isle-dc#228 being merged I suspect that these documentation change can be merged now.

@misilot I assume that are no changes needed to your docs and they can go in as is since the Islandora-Devops/isle-dc#228 merge ?

@misilot
Copy link
Contributor Author

misilot commented Apr 5, 2022

@ysuarez, yes I believe it is good to go. The PR was merged as is without any changes.

Thank you!

With the additions from Islandora-Devops/isle-dc#228 requesting
certificates via ACME / Let's Encrypt has changed, and should be easier
for users to utilize.
@ysuarez ysuarez merged commit ce98f74 into Islandora:main Apr 5, 2022
@ysuarez
Copy link
Contributor

ysuarez commented Apr 5, 2022

No, thank you @misilot

@misilot misilot deleted the acme-update branch April 5, 2022 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants